Skip to content

試験的用にカナリア版はAndroid 16でもバックグラウンド音声を有効にできるように実装#5357

Merged
TinyKitten merged 1 commit intodevfrom
feature/enable-bg-tts-canary
Feb 19, 2026
Merged

試験的用にカナリア版はAndroid 16でもバックグラウンド音声を有効にできるように実装#5357
TinyKitten merged 1 commit intodevfrom
feature/enable-bg-tts-canary

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 19, 2026

Summary by CodeRabbit

リリースノート

  • Bug Fixes
    • Android 16以上でのバックグラウンドTTS機能の動作条件を改善しました。

@TinyKitten TinyKitten self-assigned this Feb 19, 2026
@github-actions github-actions bot added the react label Feb 19, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

📝 Walkthrough

ウォークスルー

isDevAppインポートを追加し、Android 16以上のバックグラウンドTTS無効化チェックを開発アプリの実行を除外するように更新しました。isAndroid16OrHigher!isDevApp && Platform.OS === 'android' && Number(Platform.Version) >= 36に変更され、開発環境でのバックグラウンドTTS自動無効化を防ぎます。

変更

Cohort / File(s) 概要
バックグラウンドTTS開発環境チェック
src/screens/TTSSettings.tsx
isDevAppのインポートを追加し、Android 16+のバックグラウンドTTS無効化ロジックに開発アプリ判定条件を追加。開発環境での自動無効化を防止。

推定コードレビュー工数

🎯 1 (Trivial) | ⏱️ ~3 minutes

関連する可能性のあるPR

推奨ラベル

android, react

ポエム

🐰 背景に隠れし開発の道を切り分け、
Androidの森でテスト走らすため、
isDevAppの判定で別れ道を引く、
開発の自由よ、本番の堅さよ。

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルは変更内容と完全に一致しており、開発環境(カナリア版)でAndroid 16+においてバックグラウンドTTSを有効にするという主要な変更を正確に説明しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/enable-bg-tts-canary

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/screens/TTSSettings.tsx`:
- Around line 117-121: コメントが実際のロジックと合っていないので、isAndroid16OrHigher の説明を「Android 16
(API 36) ではバックグラウンド音声再生が制限されるため無効化」とだけ書くのではなく、現在の条件が dev/canary
を除外するロジックになっていることを明記してください;具体的には isAndroid16OrHigher
を定義しているコメントの前後(該当する2箇所)を更新して、Platform.OS と Number(Platform.Version) のチェックに加え
isDevApp を使って dev/canary を除外している旨を簡潔に説明し、混乱を避けるために「dev/canary
を除外している」ことを明記してください。

@TinyKitten TinyKitten merged commit b8654c0 into dev Feb 19, 2026
7 checks passed
@TinyKitten TinyKitten deleted the feature/enable-bg-tts-canary branch February 19, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant